Skip to content

Conversation

@devversion
Copy link
Member

Recently the align input binding has been removed in favor of the CSS property text-align.

If a developer sets the text-align property to end the text will start from the end and also the input underline will show incorrectly start from the end.

Setting the absolute positioned underline to left: 0 ensures that the underline always shows-up correctly (because the mat-input-underline is inside of a relative container)

Note: This still works with RTL because the mat-input-underline container is inside of a relative container and it uses a width of 100%)

Fixes #5272

Recently the `align` input binding has been removed in favor of the CSS property `text-align`.

If a developer sets the `text-align` property to `end` the text will start from the end and also the input underline will show incorrectly start from the end.

Setting the absolute positioned underline to `left: 0` ensures that the underline always shows-up correctly (because the `mat-input-underline` is inside of a relative container)

Fixes angular#5272
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Jun 21, 2017
@mmalerba mmalerba added pr: lgtm action: merge The PR is ready for merge by the caretaker and removed pr: needs review labels Jun 21, 2017
@jelbourn jelbourn merged commit 5c9391d into angular:master Jun 22, 2017
@devversion devversion deleted the fix/input-text-align-underline branch November 11, 2017 10:23
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(input-demo): remove align input binding

4 participants